Skip to content

Conversation

@gaganis
Copy link
Contributor

@gaganis gaganis commented Jun 26, 2025

Thank you for taking time to contribute this pull request!
You might have already read the [contributor guide][1], but as a reminder, please make sure to:

  • Sign the contributor license agreement
  • Rebase your changes on the latest main branch and squash your commits
  • Add/Update unit tests as needed
  • Run a build and make sure all tests pass prior to submission

@sunyuhan1998
Copy link
Contributor

Please sign off your commits to fix the DCO check.

@gaganis gaganis force-pushed the typo_in_token_documentation branch from 1fd6041 to f1541c0 Compare June 26, 2025 09:31
@ilayaperumalg ilayaperumalg self-assigned this Jun 26, 2025
@ilayaperumalg ilayaperumalg added documentation Improvements or additions to documentation for: backport-to-1.0.x labels Jun 26, 2025
@ilayaperumalg ilayaperumalg added this to the 1.1.x milestone Jun 26, 2025
----

You can also set this configuration programmatically in your application code:
You can also get this configuration programmatically in your application code:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about changing get this configuration to get environment variable?

You can also get environment variable programmatically in your application code:

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dev-jonghoonpark Thanks for the review and suggestion. It would be nice to include your suggestion but the important fix is to change set -> get.

Since we haven't heard back from the author on this yet, I will go ahead and merge this PR as is for now.

@ilayaperumalg ilayaperumalg merged commit 4ec73e2 into spring-projects:main Jul 2, 2025
2 checks passed
spring-builds pushed a commit that referenced this pull request Jul 2, 2025
Fixes #3677

Signed-off-by: Giorgos Gaganis <[email protected]>
(cherry picked from commit 4ec73e2)
@ilayaperumalg
Copy link
Member

@gaganis Thanks for the PR fixing the doc.

scionaltera pushed a commit to scionaltera/spring-ai that referenced this pull request Sep 3, 2025
chedim pushed a commit to couchbaselabs/spring-ai that referenced this pull request Sep 19, 2025
Willam2004 pushed a commit to Willam2004/spring-ai that referenced this pull request Oct 11, 2025
Auto-cherry-pick to 1.0.x
Fixes spring-projects#3677

Signed-off-by: Giorgos Gaganis <[email protected]>
Signed-off-by: 家娃 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation for: backport-to-1.0.x status: backported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants